Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move CIs to use docker builds #1316

Merged
merged 3 commits into from
Sep 3, 2022
Merged

Move CIs to use docker builds #1316

merged 3 commits into from
Sep 3, 2022

Conversation

powderluv
Copy link
Collaborator

Now that #1234 has landed and anyone can run CI / Release builds locally move GHA to use the same flow.

Now that #1234 has landed and anyone can run CI / Release builds locally move GHA to use the same flow.
Copy link
Member

@sjain-stanford sjain-stanford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with some nit requests.

.github/workflows/buildAndTest.yml Outdated Show resolved Hide resolved
.github/workflows/buildAndTest.yml Outdated Show resolved Hide resolved
@powderluv
Copy link
Collaborator Author

@silvasean if you have a quick fix for #1334 lets land that and then this change. If not we can revert the last shape lib test and re-land after this lands.

I would like to try and land this today so I can tend to the CI over the long weekend in case anything needs tweaking / reverts etc.

@silvasean
Copy link
Contributor

Let's land this! The fix is in #1338 -- feel free to merge it yourself if needed.

@powderluv
Copy link
Collaborator Author

awesome thanks. will wait for it to go green and land #1338 and then land this when green.

@silvasean
Copy link
Contributor

Awesome. Thanks Anush for all the hard work here and sorry for these last minute speed bumps on the generated files!

@powderluv powderluv merged commit e6528f7 into main Sep 3, 2022
@powderluv powderluv deleted the docker-gha branch September 3, 2022 01:35
qedawkins pushed a commit to nod-ai/torch-mlir that referenced this pull request Oct 3, 2022
Signed-off-by: Whitney Tsang <whitneyt@ca.ibm.com>
qedawkins pushed a commit to nod-ai/torch-mlir that referenced this pull request Oct 3, 2022
The modification to use the builder based interface to generate Krnl loop is completed (llvm#1250, llvm#1283, llvm#1285, llvm#1292, llvm#1293, llvm#1303, llvm#1308, llvm#1314, llvm#1316, llvm#1317, llvm#1326, llvm#1403), and BuildKrnlLoop is no longer needed.

Signed-off-by: Whitney Tsang whitneyt@ca.ibm.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants